I think it needs to be made clearer that naming the loop variable CMAKE_BUILD_TYPE was important - the fact that's a reserved special cmake-variable name is the only reason it has any effect on conan_cmake_autodetect, which is rather subtle. And using those reserved names as local variables is not usually a good practice, so that's kind of sneaky.

I think it needs to be made clearer that naming the loop variable CMAKE_BUILD_TYPE was important - the fact that's a reserved special cmake-variable name is the only reason it has any effect on conan_cmake_autodetect, which is rather subtle. And using those reserved names as local variables is not usually a good practice, so that's kind of sneaky.

Hi @puetzk,
Thanks for the comments, I think you are right, I have introduced some changes to make it more explicit, passing the BUILD_TYPE to conan_cmake_autodetect so that the use of the CMake reserved variable does not lead to confusion.